Skip to content

[hotfix][example] Add missing flink-json dependency#544

Merged
xintongsong merged 1 commit intoapache:mainfrom
JinkunLiu:hotfix/example/flink-json
Feb 24, 2026
Merged

[hotfix][example] Add missing flink-json dependency#544
xintongsong merged 1 commit intoapache:mainfrom
JinkunLiu:hotfix/example/flink-json

Conversation

@JinkunLiu
Copy link
Contributor

Linked issue: #540

Purpose of change

Add flink-json dependency for reading json at example.

Tests

  • ./tools/build.sh -j passes
  • ./tools/lint.sh passes

API

No public API changes.

Documentation

  • doc-needed
  • doc-not-needed
  • doc-included

@github-actions github-actions bot added priority/major Default priority of the PR or issue. fixVersion/0.3.0 The feature or bug should be implemented/fixed in the 0.3.0 version. doc-not-needed Your PR changes do not impact docs labels Feb 18, 2026
@JinkunLiu JinkunLiu force-pushed the hotfix/example/flink-json branch from 665cb73 to 95cb4c2 Compare February 18, 2026 13:46
@JinkunLiu JinkunLiu force-pushed the hotfix/example/flink-json branch from 95cb4c2 to 0cd2325 Compare February 18, 2026 13:47
@JinkunLiu
Copy link
Contributor Author

Hi @weiqingy, could you take a look? Thanks!

Copy link
Collaborator

@weiqingy weiqingy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Contributor

@xintongsong xintongsong left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for reporting and fixing this, @JinkunLiu . And thanks for the review, @weiqingy .

LGTM. Merging.

@xintongsong xintongsong merged commit 96ecd9c into apache:main Feb 24, 2026
39 of 40 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

doc-not-needed Your PR changes do not impact docs fixVersion/0.3.0 The feature or bug should be implemented/fixed in the 0.3.0 version. priority/major Default priority of the PR or issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants